add attribute docs #2
Conversation
luca3s
force-pushed
the
docs-attr
branch
2 times, most recently
from
f8e534c to
304b1d7
Compare
|
I have reworked it a bit to remove "disable". I think i either addressed you comments or removed the text they were part of. I hope you are fine with exposing the disable functions for now. This isn't going on stable rust, so they could be removed once the attribute exists. |
| from a function not sanitized. So a function could be both sanitized and not sanitized in one program execution. | ||
| The sanitizer can be disabled using the external function `__rtsan_disable()`. It can be enabled again using | ||
| `__rtsan_enable()`. Entering a real-time context while the sanitizer is enabled is not possible. |
There was a problem hiding this comment.
Entering a real-time context while the sanitizer is enabled is not possible.
I think you meant to say the opposite here but, regardless, I think it's possibly not worth including - IMHO it's more confusing than helpful for the average user who isn't aware of the sanitizer's inner workings. If you really feel it should be included, I'd suggest maybe something like Entering a real-time context while the sanitizer is disabled does not reactivate the sanitization.
There was a problem hiding this comment.
Yeah i thought about that too, will remove it.
Is there a open llvm/clang (or even in one of your repos) about it?
Just that it's written down at least somewhere.
There was a problem hiding this comment.
Thanks for the reminder! Yes - just added one
Great - I think your changes are nice improvements! I've left a couple more minor comments but other than those it LGTM 👍 |
luca3s
force-pushed
the
sanitize-off-on
branch
from
9d0f497 to
90c564e
Compare
luca3s
force-pushed
the
sanitize-off-on
branch
from
90c564e to
95bdb34
Compare
luca3s
force-pushed
the
docs-attr
branch
2 times, most recently
from
1cdd36f to
7ea4dc6
Compare
…=jieyouxu Rehome 30 `tests/ui/issues/` tests to other subdirectories under `tests/ui/` [#1 of Batch #2] Part of rust-lang#133895 Methodology: 1. Refer to the previously written `tests/ui/SUMMARY.md` 2. Find an appropriate category for the test, using the original issue thread and the test contents. 3. Add the issue URL at the bottom (not at the top, as that would mess up stderr line numbers) 4. Rename the tests to make their purpose clearer Inspired by the methodology that `@Kivooeo` was using. r? `@jieyouxu`
…=jieyouxu Rehome 30 `tests/ui/issues/` tests to other subdirectories under `tests/ui/` [#2 of Batch #2] Part of rust-lang#133895 Methodology: 1. Refer to the previously written `tests/ui/SUMMARY.md` 2. Find an appropriate category for the test, using the original issue thread and the test contents. 3. Add the issue URL at the bottom (not at the top, as that would mess up stderr line numbers) 4. Rename the tests to make their purpose clearer Inspired by the methodology that `@Kivooeo` was using. r? `@jieyouxu`
…r=jieyouxu Rehome 30 `tests/ui/issues/` tests to other subdirectories under `tests/ui/` [rust-lang#3 of Batch #2] Part of rust-lang#133895 Methodology: 1. Refer to the previously written `tests/ui/SUMMARY.md` 2. Find an appropriate category for the test, using the original issue thread and the test contents. 3. Add the issue URL at the bottom (not at the top, as that would mess up stderr line numbers) 4. Rename the tests to make their purpose clearer Inspired by the methodology that `@Kivooeo` was using. r? `@jieyouxu`
No description provided.